build: update dependency sass_api to v17.5.0#3587
Conversation
See associated pull request for more information.
angular-robot
added
action: merge
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the sass and sass_api dependencies to versions 1.99.0 and 17.5.0, respectively. The feedback suggests using caret syntax for the sass_api dependency in pubspec.yaml to follow Dart best practices and maintain consistency with other dependencies in the project.
| # Add regular dependencies here. | ||
| dependencies: | ||
| sass_api: 17.4.0 | ||
| sass_api: 17.5.0 |
There was a problem hiding this comment.
The sass_api dependency is pinned to an exact version, which is inconsistent with other dependencies in this file (like sprintf) that use caret syntax (^). In Dart, it is generally recommended to use semantic versioning ranges (e.g., ^17.5.0) in pubspec.yaml to allow for compatible patch and minor updates, while relying on pubspec.lock to ensure exact version pinning for reproducibility. This also helps avoid potential version solving conflicts for consumers of this package.
sass_api: ^17.5.0
|
This PR was merged into the repository. The changes were merged into the following branches:
|
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
2 participants
This PR contains the following updates:
17.4.0→17.5.0